tools/oxenstored: Fix incorrect scope after an if statement
authorAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 11 Nov 2022 18:50:34 +0000 (18:50 +0000)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Tue, 20 Dec 2022 13:13:40 +0000 (13:13 +0000)
commit2e8d7a08bcd111fe21569e9ace1a047df76da949
tree1af63a565f22832d25457d69df51aefa32b4d75f
parenta7a26da0b59da7233e6c6f63b180bab131398351
tools/oxenstored: Fix incorrect scope after an if statement

A debug statement got inserted into a single-expression if statement.

Insert brackets to give the intended meaning, rather than the actual meaning
where the "let con = Connections..." is outside and executed unconditionally.

This results in some unnecessary ring checks for domains which otherwise have
IO credit.

Fixes: 42f0581a91d4 ("tools/oxenstored: Implement live update for socket connections")
Reported-by: Edwin Török <edvin.torok@citrix.com>
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Christian Lindig <christian.lindig@citrix.com>
(cherry picked from commit ee36179371fd4215a43fb179be2165f65c1cd1cd)
tools/ocaml/xenstored/xenstored.ml